Skip to content

[codex] raise Rust coverage above 90 percent#2

Merged
stephenlclarke merged 2 commits into
mainfrom
develop
Apr 11, 2026
Merged

[codex] raise Rust coverage above 90 percent#2
stephenlclarke merged 2 commits into
mainfrom
develop

Conversation

@stephenlclarke
Copy link
Copy Markdown
Owner

Summary

  • add targeted coverage for validator, obfuscator, and FIX helper paths that were still under-covered
  • exercise CLI info/message/tag/component handlers end to end so src/main.rs and display rendering paths are covered through real command flows
  • add generator and pcap2fix tests for repo-root discovery, sensitive-tag generation, resync, overlap, overflow, and other edge cases

Why

SonarCloud coverage on main was still below the desired threshold because several non-streaming CLI paths, the sensitive-tag generator, and pcap2fix error branches had little or no test coverage.

Impact

  • raises the Sonar-aligned Cobertura report to 92.21% overall Rust line coverage
  • improves regression protection around dictionary browsing, FIX helper behavior, generated sensitive-tag output, and TCP FIX reassembly edge cases

Validation

  • cargo test --workspace
  • cargo clippy --workspace --all-targets -- -D warnings
  • make coverage

@stephenlclarke stephenlclarke marked this pull request as ready for review April 11, 2026 17:06
@sonarqubecloud
Copy link
Copy Markdown

@stephenlclarke stephenlclarke merged commit 14e15f0 into main Apr 11, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant